Skip to content

JAVA check for DRIVERS-3547: Explicit Encryption Case 2 removes contention-0 find step#2002

Closed
rozza wants to merge 2 commits into
mongodb:mainfrom
rozza:DRIVERS-3547
Closed

JAVA check for DRIVERS-3547: Explicit Encryption Case 2 removes contention-0 find step#2002
rozza wants to merge 2 commits into
mongodb:mainfrom
rozza:DRIVERS-3547

Conversation

@rozza

@rozza rozza commented Jun 16, 2026

Copy link
Copy Markdown
Member

Servers implementing SERVER-91887 require find payloads to exactly match the collection's configured contention factor, not just avoid exceeding it. Drop the find with contentionFactor=0 (which the server now rejects) and recreate the collection with contention=10 before inserting.

@rozza rozza changed the title DRIVERS-3547: Explicit Encryption Case 2 removes contention-0 find step JAVA check for DRIVERS-3547: Explicit Encryption Case 2 removes contention-0 find step Jun 17, 2026
rozza added 2 commits June 17, 2026 12:01
Servers implementing SERVER-91887 require find payloads to exactly match
the collection's configured contention factor, not just avoid exceeding it.

Following specifications PR mongodb#1952, set up a pre-built
db.explicit_encryption_c10 collection (contention 10) via the new
encryptedFields-c10.json fixture, and have Case 2 insert into and find
against it with contentionFactor 10.
…VERS-3547

TODO DRIVERS-3547: revert this commit once specifications PR mongodb#1952
(mongodb/specifications#1952) is merged. Point the
submodule URL back at https://github.com/mongodb/specifications master and
update the gitlink accordingly.

Updates both the .gitmodules url (so fresh clones/CI fetch from the fork)
and the gitlink SHA. Provides the encryptedFields-c10.json fixture and
updated README required by the Explicit Encryption Case 2 test changes.
@rozza

rozza commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

Failing tests are addressed in #2001

@rozza rozza closed this Jun 18, 2026
@rozza rozza deleted the DRIVERS-3547 branch June 18, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant